-
Notifications
You must be signed in to change notification settings - Fork 364
re-enable unit and full-app tests #172
base: master
Are you sure you want to change the base?
Conversation
Any test case that has the @watch flag will not be run by chimp in CI mode. These should only be used during development.
…acceptance-test Chore/remove watch flag from acceptance test
How did you get the file permissions to work on circlecl? I'm trying to update the tests in the coffeescript branch, and am running into this. I assume I should be modifying the pre section of the circle.yml? |
Hi @noahsw - thanks for the PR. Are you still working on this? Let me know if you are, or if you'd like me to take a look. |
Hi @hwillson - sorry I ran into CI problems and had to take a break. Maybe this is why the tests were removed in the first place... I hope someone can figure it out though! |
No problem @noahsw - I'll take a look. Thanks again! |
See also #210 |
|
Tests were removed from CI in #164 but they're working fine for me on master so we should add them back.
Open question is whether we should be running
meteor npm install
ornpm install
, but let's see if CI passes.